Skip to content

DATAREDIS-551 - Fix pageable query execution when derived criteria is empty. #220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

mp911de
Copy link
Member

@mp911de mp911de commented Sep 12, 2016

We now make sure to count records without using criteria when derived criteria is empty. This allows usage of declared query methods using Pageable without criteria like findBy(Pageable page).


Related ticket: DATAREDIS-551

… empty.

We now make sure to count records without using criteria when derived criteria is empty. This allows usage of declared query methods using `Pageable` without criteria like `findBy(Pageable page)`.
christophstrobl pushed a commit that referenced this pull request Sep 13, 2016
… empty.

We now make sure to count records without using criteria when derived criteria is empty. This allows usage of declared query methods using `Pageable` without criteria like `findBy(Pageable page)`.

Original Pull Request: #220
christophstrobl added a commit that referenced this pull request Sep 13, 2016
Fix indentation.

Original Pull Request: #220
@christophstrobl christophstrobl deleted the issue/DATAREDIS-551 branch September 13, 2016 08:26
christophstrobl pushed a commit that referenced this pull request Sep 13, 2016
… empty.

We now make sure to count records without using criteria when derived criteria is empty. This allows usage of declared query methods using `Pageable` without criteria like `findBy(Pageable page)`.

Original Pull Request: #220
christophstrobl added a commit that referenced this pull request Sep 13, 2016
Fix indentation.

Original Pull Request: #220
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants